broadway: Define variables where they are used

Avoids gcc warnings about unused variables.

composetable: Fix gcc warnings

immodule: Use correct sign for function prototype

entry: Turn progress node into a gadget

entry: Simplify drawing code

Adwaita: fix headerbar spinbutton size

fixes https://bugzilla.gnome.org/show_bug.cgi?id=761484

gdkborder: remove unneeded definition

This reverts 54fa65f544bef281640d18a4a725005f6199a956,
5190d010d95010a845715c719c3b85a75e0c513a, and
a6e4de28842002e7bfdf73def26c4c52d9f9126a.

https://bugzilla.gnome.org/show_bug.cgi?id=756579

print dialog: Fix layout preview drawing

Use the CSS name paper for this as well, and make sure Adwaita
sets a color that makes the text visible in the dark theme as well.

Adwaita: notebook with hidden header style fix

the stack styling is applied only when there is a header, hence
when the stack node is not the notebook only child, should cure
https://bugzilla.gnome.org/show_bug.cgi?id=760996

Adwaita: fix vertical spinbutton style

correct sizing and workaround disabled entry text colors.

notebook: Do not reorder tabs after a grab notify if there isn't an ongoing reorder operation

This broke in f535251507a6a0af68aed6348ae80cbdac9b1f4a that removed
during_reorder in favor of using the current operation, but removed the
check from gtk_notebook_stop_reorder().

Updated French translation
(cherry picked from commit 85fb4c1c0a17906a65e0b732770b88353b6214c4)

gtksettings: reset property to correct default value

https://bugzilla.gnome.org/show_bug.cgi?id=76138

Revert "Add missing (out) annotation to gdk_rgba_parse"

This reverts commit 695860958a9a2e7c825bcec4d0d42b427f0fb47e.

The change introduced an introspection ABI break.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=682125

If we want to move the parse() method to be a constructor or a class
method, we'll need to do more testing or add new API.

https://bugzilla.gnome.org/show_bug.cgi?id=754990

entry: Draw progress in the right place

Updated Hungarian translation

Updated German translation

reftests: Make separator-size test pass in HighContrastInverse

After neutralizing the stack background, the notebook itself
was peeking through.

Updated Spanish translation

Updated Spanish translation

Trivial formatting fix

reftests: Make label-text-shadow-changes-modify-clip test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the rui on the fly if we find that animations
are disabled.

reftests: Add a message to the test output

Emit a message that ends up in the test output if we are modifying
the test on the fly.

reftests: Rename a source file

Easier to keep track of things if the sources are names after
the test they are used in.

entry: Fix icon areas

Do the same thing we already do in gtk_entry_get_text_area.

widget: Annotate out params of _translate_coordinates as optional

reftests: Make animation-direction test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the reference ui on the fly if we find
that animations are disabled.

gtk-builder-tool: Expand the preview command

Make the preview command parse options properly, turn the ID into
an --id=ID option, and add a --css=FILE option that allows to
specify a css file to use for previewing.

This is useful for e.g. previewing the reftest .ui files with
their corresponding .css.

gtk-launch: Add --version

Support --version. Also improve the --help output cosmetically.

gtk-builder-tool: Accept --help

Saying 'Failed to open file '--help' is not polite or smart.
Catch this case and show usage information.

file chooser: Avoid a crash

Since 39c2d12330b6d4405ca8a5599c12017c58626fcf,
priv->operation_mode == OPERATION_MODE_BROWSE no longer
guarantees that priv->browse_files_model is the current
model of the list - we are only switching the models after
loading the new directory. Avoid triggering the assertion
in show_and_select_files by checking if we have right model
before calling it.

https://bugzilla.gnome.org/show_bug.cgi?id=761209

Don't include the builtin sequence table in multiple places

The table is defined as static const guint16 [], so it really
should only be included in one source file.

combobox: Avoid a misleading indentation

gcc 6 warns about this sort of thing. There were also some
stray ; here.

Fix the border-image-excess-size reftest

This was failing because the grid is now respecting border
width too. Making the selector more specific so it only
applies to the toolbars fixes this.

Fix another reftest

Make sure the notebook in the separator-size reftest does not
peek through, by giving it a transparent background.

Fix some reftests under wayland

Tests need to use popup windows, in order to avoid differences
due to CSD.

wayland: Reduce warning level for parentless temporaries

Temp windows without parent are used frequently in our testsuite;
using g_warning on them causes the tests to fail, which is not
useful. Reduce the warning to a g_message.

treeview: Draw background when empty

We have a function we call for drawing the empty case, but
it only renders a focus rectangle. Make it draw the background
as well.

https://bugzilla.gnome.org/show_bug.cgi?id=761309

colorchooserwidgeet: Reset the selected swatch when removing

Otherwise, priv->current points to garbage and subsequently leads to a
crash, e.g. when adding another custom color.

Document gtk_parse_args limitations

Mention that some things won't work without a display
connection.

Throw an error when a display is missing

Creating style context won't work without a display, so
give a clear error message and abort instead of segfaulting
later on.

https://bugzilla.gnome.org/show_bug.cgi?id=761332

3.19.8

Remove entry-progress-coloring reftest

We don't recolor text over progress anymore, so this
tests a no longer existing feature.

Use TAP output for installed tests

Run reftests with HighContrastInverse too

For installed tests, we run the reftests with all the builtin
themes (minus platform-specific ones). Add HighContrastInverse
to that list.

Adwaita: Add symbolic png assets

And use them as fallback for the svg ones. This lets us
avoid a hard dependency on librsvg - the theme will work
without svg support.

icontheme: Avoid a crash

gtk_icon_info_new_for_file with a size of 0 would fail for
symbolic pngs because we ended up scaling with 0. Avoid this.

css: Support the image() notation

This lets us do fallback in case an image format is not
supported, and also lets us provide solid-color images.
We don't support image fragment notations.

See ttps://www.w3.org/TR/css3-images/#image-notation

https://bugzilla.gnome.org/show_bug.cgi?id=761318

Emit an error if image loading fails

Instead of spamming stderr with g_warning, use the new
emit_error method of the GtkStyleProviderPrivate interface
to emit an error if loading an image fails.

CSS provider: Add a way to emit errors

Currently, GtkCssProvider can emit ::parsing-error only during
the actual parsing, although the documentation hints that it might
happen at other times.

This commit adds a emit_error method to the GtkStyleProviderPrivate
interface that will let us emit errors from the compute() implementations
as well, which can be useful (e.g. if an image fails to load).

Updated Hungarian translation

Updated Hungarian translation

Updated Hungarian translation

file chooser: Make Escape close the dialog

In some situations (no header bar, save mode), hitting Escape
would not do anything because the entry ate the key event.
Fix this by telling the entry to only handle Escape when there
is something to do, such as switching back to the path bar.

https://bugzilla.gnome.org/show_bug.cgi?id=761026

Add a css style test for color names

Verify that we support all the css color names, with their expected
rgb values.

css image recolor: Load images at the proper scale

We should not hardcode a scale of 1, this leads to
pixellated upscaled images at scale=2, even if the source
is an svg. By passing the proper scale, we can load the
svg at the correct size.

icon theme: Improve loading of unthemed files

When creating icon info objects for unthemed files, we don't
really have a nominal size, so we pass 0 to mean 'load at
original size'. However, this is not what was happening.
To make this possible, add variants of some pixbuf loading
functions that take a scale factor instead of a desired size,
and use those when we don't have a nominal size.

image: use GtkMisc padding again

This was lost in the gadget conversion.
Thankfully, the misc-alignment reftest catches this.

Use an GtkImage instead of GtkArrow

GtkArrow is deprecated, and the recommended replacement
is GtkImage.

reftests: get icon-vfuncs test almost working again

The remaining difference here is GtkArrow, which is deprecated
and has not been updated to the latest CSS rendering standards.

Adwaita: reset padding on notebook tab close button

testgtk: Avoid deprecation warnings

Eventually, we should probably remove the examples that rely
on geometry support, since they probably don't work correctly
anymore. For now, just disable the warnings.

testoffscreen: Remove deprecated calls

Not doing these calls doesn't seem to affect the demo,
so lets just remove them.

testgeometry: Ignore deprecations

All the functionality that is tested here has been deprecated
recently.

Adwaita: fix border radius on #editor-color-sample

Adwaita: notebook style rewrite

Update gtk-builder-tool docs

builder tool: Add a preview command

This was somewhat missing, and is easy to add.

docs: Fix a duplicate id

This was causing the css properties chapter to appear twice
in the docs, and the overview not at all.

Updates

infobar: Don't expose implementation details

Only export the documented widgets as internal children.

searchbar: Don't expose implementation details

The various boxes and the revealer here should not be exposed
as internal children for subclasses to use in their .ui files.

actionbar: Don't expose implementation details

The box and revealer here should not be exposed as internal
children for subclasses to use in their .ui files.

dialog: Stop exporting action_box

This was not meant as an internal child that subclasses
can refer to in their .ui files.

https://bugzilla.gnome.org/show_bug.cgi?id=761254

print dialog: Stop using action_box in .ui

We can just use CSS margins nowadays to achieve the desired
appearance.

label: fix a case of misleading indentation

gcc 6 complains aobut things like this.

tests: Avoid deprecated API usage in testgtk

We can use seat grabs here.

tests: Avoid deprecated API usage in testsocket

We can use seat grabs here.

tests: Fix deprecation warnings in testinput

demos: Modify "Change display" demo to use seat grabs

The window picking feature was done through a pointer grab, replace
it by a seat grab on pointing devices.

gtkdnd: Fix deprecation compile warnings

Grabbing must stay a bit longer until all other backends than x11/wayland
catch up with GDK DnD, so ignore deprecation flags are used on those. The
uses of GdkDeviceManager can be entirely avoided though.

label: use GtkMisc padding again

this was lost in the gadget conversion in a81267c28e4e0368cd0e23b661e1c8c53f8194fb

https://bugzilla.gnome.org/show_bug.cgi?id=760522

spin button: Document sizing peculiarities

Document that it is best to explicitly set width-chars and
max-width-chars to avoid surprises.

https://bugzilla.gnome.org/show_bug.cgi?id=727294

composetable: Don't abort on unsupported escape sequences

People might put all sorts of gunk in their .XCompose file, in
the hope that XLib makes sense of it. Even if we don't make sense
of it, we shouldn't abort, but instead ignore the lines we can't
understand. Pointed out in
https://bugzilla.redhat.com/show_bug.cgi?id=1301254

Update README.glade

Update the README.glade file to use current .ui file
names and locations as example.

Updated German translation

css node tests: Update expected output

The entry undershoot change needs to be reflected in the
test output.

window: Always disconnect signal handlers

We connect to the titlebar widgets change notification regardless
whether it is internally created or not, so don't make the signal
handler disconnection conditional on that either.

assistant: Avoid tripping template tests

We need to unset the titlebar manually before chaining up
in destroy, otherwise we trigger the template invariant
checking - GtkWindow would eventually unset it, but too late
for the invariants checking code in gtk_widget_destroy.

Tweak startup-notification after the first window

Presently, Gtk will only send a startup notification completion message
for the first window that is shown.  This is not good for the case of
GtkApplication, where we are expected to participate in
startup-notification for all windows.

We have avoided this problem by manually emitting the startup complete
message from after_emit in GtkApplication.

Unfortunately, this causes problems for windows that are shown with a
delay.  It is also a dirty hack.

The reason for the original behaviour is simple: there is a static
boolean in gtkwindow.c which controls it.  We remove this.

Instead, clear the startup notification ID stored in GDK when sending
the completion message.  GtkApplication will re-set this the next time
an event comes in which needs startup-notification handling.  In the
non-GtkApplication case, newly shown windows will still not send the
message, since the cookie will have been cleared.

Finally, we remove the hack from GtkApplication's after_emit.

This will probably cause some regressions in terms of lingering startup
notification messages.  The correct solution here is to always use
gtk_window_present(), including when merely opening a new document (with
a new tab, for example).

https://bugzilla.gnome.org/show_bug.cgi?id=690791

modelbutton: Use indicator state for button gadget if iconic

entry: Fix a typo

entry: Fix a case of disagreeing coordinate systems

gesture_get_current_point() returned x/y in layout coordiantes, but
callers assumed it in textarea coordinates.

Adwaita: fix spinbutton styling

Spinbuttons were quite broken after the latest changes. This commit
brings the style back around what it was for 3.18.

Adwaita: Add some themeing for entry undershoot

This is just copying what we do for scrolled windows.
It will need some tweaks to look good for entries.

entry: give a visual clue that content is scrolled

This has been requested long ago, and we now have the
machinery to implement it easily.

box gadget: Improve assertion message

Include the owner of the gadget in the assertion message,
to improve the chances of tracking down whats actually
going wrong here.

color button: Allow skipping the palette

Allow skipping the palette in the dialog, and going directly
to the editor. This is useful when the color button is already
part of a palette.

Fix a typo

entry: Compute selection range correctly

gtk_editable_get_selection_bounds() returns UTF-8 character offsets,
but gdk_pango_layout_get_clip_region() wants byte ranges, so convert
from one to the other.
With English, this is especially visible for passwords, which use ●
as the invisible character.

https://bugzilla.gnome.org/show_bug.cgi?id=761128